home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / sys / amiga / programmer / 2437 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  2.8 KB
  1. Path: soap.news.pipex.net!pipex!usenet
  2. From: m.hendry@dial.pipex.com (Mathew Hendry)
  3. Newsgroups: comp.sys.amiga.programmer
  4. Subject: Re: silly c problem
  5. Date: Wed, 31 Jan 96 14:29:27
  6. Organization: Private node.
  7. Distribution: world
  8. Message-ID: <19960131.4755A8.D114@an181.du.pipex.com>
  9. References: <f2c_9601302132@techtol.magic.mb.ca>
  10. NNTP-Posting-Host: an181.du.pipex.com
  11. X-Newsreader: TIN [AMIGA 1.3 950726BETA PL0]
  12.  
  13. The problem here is the use of pointers in printf() - integer arguments
  14. should _not_ normally be referenced with pointers when calling printf(). If
  15. you do so, what will be printed, logically enough, is the address at which the
  16. integer is stored, rather than the value of that integer.
  17.  
  18. Perhaps you are confused by the different calling conventions of printf() and
  19. scanf() - it is safe to pass by value to printf(), since the value does not
  20. need to be changed by printf(). scanf(), on the other hand, will alter the
  21. value and so must be passed a _pointer_ to that value.
  22.  
  23. Tahir Khawaja (tahir.khawaja@techtol.magic.mb.ca) wrote:
  24. : #include <stdio.h>
  25. : 
  26. : main()
  27. : {
  28. :   int num1, num2, res, ope, err;
  29. : 
  30. :   printf("Please input two number!\n");
  31. :   scanf("%d%d", &num1, &num2);
  32. :   printf("You entered %d and %d.\n\n", &num1, &num2);
  33.  
  34.     printf("You entered %d and %d.\n\n", num1, num2);
  35.  
  36. :   printf("And now the code for the operation!\n");
  37. :   printf("1=Add, 2=Subtract, 3=Multiply, 4=Divide\n");
  38. :   scanf("%d", &ope);
  39. :   printf("You entered %d.\n\n", &ope);
  40.  
  41.     printf("You entered %d.\n\n", ope);
  42.  
  43. :   err = 1;
  44. :   if(ope == 1)
  45. :     {
  46. :       res = num1 + num2;
  47. :       err = 0;
  48. :     }
  49. :   if(ope == 2)
  50. :     {
  51. :       res = num1 - num2;
  52. :       err = 0;
  53. :     }
  54. :   if(ope == 3)
  55. :     {
  56. :       res = num1 * num2;
  57. :       err = 0;
  58. :     }
  59. :   if(ope == 4)
  60. :     {
  61. :       res = num1 / num2;
  62. :       err = 0;
  63. :     }
  64. :   if(err == 1)
  65. :     printf("Wrong Code! Input only number 1 - 4!\n");
  66. :   else
  67. :     printf("The result is %d %d %d %d\n.", &num1, &num2, &res, &err);
  68.  
  69.       printf("The result is %d %d %d %d\n.", num1, num2, res, err);
  70.  
  71. : }
  72. : -------------------------------
  73. : 
  74. :      pretty simple right?
  75. : 
  76. : --------------------------------
  77. : Please input two number!
  78. : 2 2
  79. : You entered 2817020 and 2817016.
  80.  
  81. These are the _addresses_ of the two values entered.
  82.  
  83. : And now the code for the operation!
  84. : 1=Add, 2=Subtract, 3=Multiply, 4=Divide
  85. : 1
  86. : You entered 2817008.
  87.  
  88. Same thing again.
  89.  
  90. : The result is 2817020 2817016 2817012 2817004
  91. : ----------------------------------
  92.  
  93. Ditto.
  94.  
  95. :      so what am i doing wrong.  i'm an absolute beginner at c but not at
  96. : programming.  the books i have don't tell me any different way to do this.  im
  97. : compiling w/ 'gcc -noixemul -o math math.c'  also pretty simple.  do i need to
  98. : #include some more stuff?
  99.  
  100. No.
  101.  
  102. Hope this helps (TM).
  103.  
  104. -- Mat.
  105.